Skip to content

Remove dead RHEL 7/8 ansible code paths and orphaned nutanix cloud-init files#2001

Open
mboersma wants to merge 1 commit into
kubernetes-sigs:mainfrom
mboersma:cleanup-orphaned-rhel8-photon3-dirs
Open

Remove dead RHEL 7/8 ansible code paths and orphaned nutanix cloud-init files#2001
mboersma wants to merge 1 commit into
kubernetes-sigs:mainfrom
mboersma:cleanup-orphaned-rhel8-photon3-dirs

Conversation

@mboersma
Copy link
Copy Markdown
Contributor

@mboersma mboersma commented May 8, 2026

Change description

Cleanup of dead code and orphaned files for RHEL/CentOS 7 and 8 paths that are no longer build targets. The Makefile and README matrix have only *-9 variants for RHEL/CentOS/Rocky/AlmaLinux.

Removes:

  • roles/providers/tasks/main.yml: the Set cloudinit feature flags for redhat 8 task (gated on distribution_major_version == "8").
  • roles/sysprep/tasks/main.yml: the <= 7 guard around the Rotate journalctl task (always-true now).
  • packer/nutanix/linux/cloud-init/{rhel,rockylinux}/8/user-data.tmpl: orphaned — there's no rhel-8.json or rockylinux-8.json in packer/nutanix/ that would consume them.

make lint passes cleanly.

  • Is this change including a new Provider or a new OS? (y/n) n

Related issues

Additional context

No behavior change for any currently-built distro.

…it files

The repo no longer builds RHEL/CentOS 7 or 8 (only -9 variants are in
the Makefile and README matrix), so several conditional tasks and a
pair of orphaned Nutanix cloud-init templates are unreachable:

- roles/providers/tasks/main.yml: 'Set cloudinit feature flags for
  redhat 8' (gated on RHEL 8).
- roles/sysprep/tasks/main.yml: 'not (RedHat and major_version <= 7)'
  guard around journalctl rotate.
- packer/nutanix/linux/cloud-init/{rhel,rockylinux}/8/user-data.tmpl:
  no rhel-8.json or rockylinux-8.json consumes them.
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jsturtevant for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 8, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@mboersma: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-ova-all 2b67324 link false /test pull-ova-all

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@tuxtof
Copy link
Copy Markdown
Contributor

tuxtof commented May 9, 2026

Hello @mboersma , we have RHEL8 target again, did i miss something ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants